[Infra] - ci/cd, ec2, alb, mysql, logging, monitoring, nginx by kargowild · Pull Request #5 · samhap-soft/kokomen-notification

kargowild · 2025-07-19T05:58:21Z

prod ec2 2대 배포 완료
prod용 alb 구성 완료 (health check에서 오류나는 중)
- 멘토링 받은 내용대로 API Gateway로 전환과정 필요
prod용 mysql ec2 실행 완료
- mysql 컨테이너 띄우기 까지만 완료
nginx
- dev용 nginx는 kokomen-backend 레포지토리에서 nginx.conf 변경만 진행 완료 -> 머지 후 nginx 컨테이너 재시작 필요
ci/cd
1. dev internal ci/cd는 정상 작동하는 것 확인함
2. dev api ci/cd도 정상 작동하는지 확인하려면 api 폴더 쪽에 변경사항 있는 PR 올려봐야 함
3. internal, api 모두 변경사항이 있을 경우 ci/cd가 둘 다 잘 도는지도 확인 필요 (2번 확인되면 아마 잘 돌 듯)
logging, prod용 ci/cd, prod용 nginx 설정은 새로운 이슈에서 진행 예정

github-actions · 2025-07-19T05:59:55Z

Test Results

1 tests 1 ✅ 0s ⏱️
1 suites 0 💤
1 files 0 ❌

Results for commit 7615f0e.

♻️ This comment has been updated with latest results.

kargowild

+) file changed에 없어서 코멘트를 못 달았는데,

internal의 application.yml 에서 #local profile에 해당하는 내용은 어떤 역할인가요? 현재 필요없는 상태가 맞을까요?!

kargowild · 2025-07-20T04:10:43Z

.github/workflows/cd-api-dev.yml

+jobs:
+  detect-changes:
+    runs-on: ubuntu-latest
+    outputs:
+      api_changed: ${{ steps.filter.outputs.api }}
+    steps:
+      - uses: actions/checkout@v4
+      - id: filter
+        uses: dorny/paths-filter@v3
+        with:
+          filters: |
+            api:
+              - 'api/**'


internal, api용 브랜치를 나누지 않기로 했으니, 이전에 논의했던 대로 파일 변경이 생긴 경우에만 trigger되도록 cd 파일을 분리했습니다.

찾아보니까 특정 폴더에 파일 변경이 생겼는지 알아보기 위해 dorny/paths-filter라는 액션을 많이 사용하길래 적용해봤습니다!

오 확정이었을까요??

전에 말씀드린 것처럼 dev -> prod 로 merge 될때 각 모듈이 따로 배포 될 수 있도록 조절이 가능한 것일까요?

아 그 부분을 놓치고 구현했네요 😅
예를 들어 develop에는 api, internal 변경사항이 모두 생겼는데 api만 배포하고 싶은 상황 말씀하시는 거 맞을까요?
저번에 질문 드리다가 정리가 안됐었는데, 이 부분을 멘토님께 정리해서 질문드려봐야 할 것 같네요!!

이거 혹시 좀 더 구체적인 상황을 예시들어주실 수 있나요? 제가 좀 헷갈려서요 ㅠ

kargowild · 2025-07-20T04:11:32Z

.github/workflows/cd-api-dev.yml

+    outputs:
+      api_changed: ${{ steps.filter.outputs.api }}


이게 변수처럼 사용돼서 아래 build-api를 돌릴지 말지 결정하게 됩니다.

kargowild · 2025-07-20T04:14:26Z

.github/workflows/cd-api-dev.yml

+          username: ${{ secrets.DOCKER_USERNAME }}
+          password: ${{ secrets.DOCKER_PASSWORD }}


이런 시크릿변수들 너무 공용으로 사용하니까 레포지토리마다 선언하는게 아니고 Organization secrets로 설정했습니다!
아직 기존 kokomen-backend에서는 Repository secrets를 사용하고 있는데, 겹치는 변수들은 삭제해도 될 것 같아요.

네네 좋습니다!! 이슈, PR 템플릿도 organization 기준으로 설정 가능하던데, 고것도 한번 참고해서 적용하면 좋을 거 같네요!

kargowild · 2025-07-20T04:16:00Z

docker/dev/docker-compose-dev.yml

+    expose:
+      - 8080


모든 컨테이너에서 expose 적극 활용했습니다. 모듈마다 포트번호 따로 지정해주는게 번거로울 것 같아서요!

kargowild · 2025-07-20T04:16:56Z

docker/dev/docker-compose-dev.yml

+    restart: always
+    environment:
+      TZ: Asia/Seoul
+      MYSQL_USER: ${SPRING_DATASOURCE_USERNAME_DEV}


헷갈리는게 ${SPRING_DATASOURCE_USERNAME_DEV} 이거 우리 kokomen으로 쓰고 있었나요 root로 쓰고 있었나요? 일단 root로 해서 잘 돌아가는 것까진 확인했습니다

저도 잘 기억이 안나네요 ㅎ..

kargowild · 2025-07-20T04:18:07Z

docker/dev/docker-compose-dev.yml

+    networks:
+      - dev-kokomen-net
+
+  notification-mysql-dev-exporter:


prometheus는 kokomen-backend의 docker compose에서 돌리고 있어서, 거기서 위 exporter도 바라보도록 수정해줬습니다. (PR만 올라간 상태)

도커네트워크를 싹다 연결해놔서 아마 잘 찾아갈 것 같습니다!

kargowild · 2025-07-20T04:18:38Z

.github/workflows/cd-internal-dev.yml

+        run: |
+          export HOSTNAME=$(hostname)
+          cd kokomen-notification/docker/dev
+          sudo -E docker compose -f docker-compose-dev.yml up -d kokomen-notification-dev-internal


-d 뒤에 애플리케이션 컨테이너만 실행하도록 구체적으로 명시했습니다

kargowild · 2025-07-20T04:20:41Z

domain/src/main/resources/application-domain.yml

+    activate:
+      on-profile: dev
+  datasource:
+    url: jdbc:mysql://kokomen-notification-mysql-dev:3306/kokomen-notification-dev


이거 이렇게 컨테이너 이름으로 박아놔도 되겠죠?
kokomen-backend에서는 이걸 왜 시크릿으로 관리했었는지 기억이 잘 안나네요

네 dev라 괜찮아 보입니다 ㅎㅎ

nak-honest

브랜치 쪼개는 쪽 이야기 한번 더 해보고 merge 시키면 좋을 것 같아 일단 comment로 남겨드려요!
고생 많으셨습니다~!!

nak-honest · 2025-07-21T07:22:28Z

.github/workflows/cd-api-dev.yml

+jobs:
+  detect-changes:
+    runs-on: ubuntu-latest
+    outputs:
+      api_changed: ${{ steps.filter.outputs.api }}
+    steps:
+      - uses: actions/checkout@v4
+      - id: filter
+        uses: dorny/paths-filter@v3
+        with:
+          filters: |
+            api:
+              - 'api/**'


오 확정이었을까요??

전에 말씀드린 것처럼 dev -> prod 로 merge 될때 각 모듈이 따로 배포 될 수 있도록 조절이 가능한 것일까요?

nak-honest · 2025-07-21T07:24:20Z

.github/workflows/cd-api-dev.yml

+          username: ${{ secrets.DOCKER_USERNAME }}
+          password: ${{ secrets.DOCKER_PASSWORD }}


네네 좋습니다!! 이슈, PR 템플릿도 organization 기준으로 설정 가능하던데, 고것도 한번 참고해서 적용하면 좋을 거 같네요!

nak-honest · 2025-07-21T07:27:42Z

.github/workflows/ci-api-test.yml

+        if: always()
+        with:
+          report_paths: ${{ github.workspace }}/api/build/test-results/**/*.xml
+          token: ${{ github.token }} 


개행문자 추가해주시면 감사하겠습니닷!

nak-honest · 2025-07-21T07:29:02Z

docker/dev/docker-compose-dev.yml

+    restart: always
+    environment:
+      TZ: Asia/Seoul
+      MYSQL_USER: ${SPRING_DATASOURCE_USERNAME_DEV}


저도 잘 기억이 안나네요 ㅎ..

nak-honest · 2025-07-21T07:30:42Z

domain/src/main/resources/application-domain.yml

+    activate:
+      on-profile: dev
+  datasource:
+    url: jdbc:mysql://kokomen-notification-mysql-dev:3306/kokomen-notification-dev


네 dev라 괜찮아 보입니다 ㅎㅎ

nak-honest · 2025-07-21T07:38:34Z

internal의 application.yml 에서 #local profile에 해당하는 내용은 어떤 역할인가요? 현재 필요없는 상태가 맞을까요?!

현재는 내용이 비어있고 domain 모듈의 로컬 프로파일을 가져오고 있어서 아무 역할을 안하고 있습니다! 처음에는 내용이 따로 있다보니 따라온 거였어요!
삭제하셔도 냅두셔도 무방합니다

kargowild added 2 commits July 18, 2025 21:03

fix: internal api에서 실수로 발생한 문제 보완

e748bec

chore: ci-internal-test 추가

6ed5066

kargowild self-assigned this Jul 19, 2025

kargowild linked an issue Jul 19, 2025 that may be closed by this pull request

[Infra] - ci/cd, ec2, alb, mysql, logging, monitoring, nginx #2

Open

kargowild added 11 commits July 20, 2025 01:29

chore: cd 파일 추가 및 dev를 위한 docker compose를 한 곳에서 관리

6a40f80

chore: cd 규칙 임시 변경

9de9168

chore: cd-dev에서 사용하는 도커컴포즈 파일명 수정

b13ee51

chore: cd-dev에서 사용하는 도커컴포즈 경로 수정

4931eaf

chore: cd-dev에서 사용하는 도커컴포즈 파일명 재수정

6a65c48

chore: cd-dev에서 발생한 문제점 보완

e150069

chore: 사용하는 도커이미지명 수정

35f5834

chore: kokomen-notification-dev-internal에서 사용하는 환경변수 추가

4ef80ab

chore: ci도 각자의 경로에 변경사항이 있을 때만 build 수행하도록 변경

6c2b6a0

chore: cd-api-dev.yml용도 미리 작성 및 cd-internal-dev.yml에서 필요없는 부분 수정

41f50aa

chore: boorJar 중복되는 설정 제거

9b900b1

kargowild commented Jul 20, 2025

View reviewed changes

nak-honest reviewed Jul 21, 2025

View reviewed changes

style: 파일 끝에 개행 문자 추가

7615f0e

kargowild merged commit 8bffcc1 into develop Jul 24, 2025
6 checks passed

		username: ${{ secrets.DOCKER_USERNAME }}
		password: ${{ secrets.DOCKER_PASSWORD }}

Comments

Conversation

kargowild commented Jul 19, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Jul 19, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results

Uh oh!

kargowild left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

kargowild Jul 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

nak-honest left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

nak-honest commented Jul 21, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

kargowild commented Jul 19, 2025 •

edited

Loading

github-actions bot commented Jul 19, 2025 •

edited

Loading

kargowild Jul 22, 2025 •

edited

Loading